OCPBUGS-77895: Allow for override of agent vcpu and memory#1862
OCPBUGS-77895: Allow for override of agent vcpu and memory#1862bfournie wants to merge 1 commit intoopenshift-metal3:masterfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
bfournie
force-pushed
the
allow-agent-overides
branch
from
40d6034 to
cd31964
Compare
|
/hold |
openshift-ci
bot
added
the
do-not-merge/hold
|
/retest |
bfournie
force-pushed
the
allow-agent-overides
branch
from
cd31964 to
0e08c59
Compare
|
/unhold |
openshift-ci
bot
removed
the
do-not-merge/hold
bfournie
force-pushed
the
allow-agent-overides
branch
4 times, most recently
from
29ce364 to
567b3d4
Compare
common.sh
Outdated
| export ARM_WORKER_MEMORY=${ARM_WORKER_MEMORY:-${WORKER_MEMORY}} | ||
| export ARM_WORKER_DISK=${ARM_WORKER_DISK:-${WORKER_DISK}} | ||
| export ARM_WORKER_VCPU=${ARM_WORKER_VCPU:-${WORKER_VCPU}} | ||
| if [[ -z ${AGENT_E2E_TEST_SCENARIO:-} ]]; then |
There was a problem hiding this comment.
It doesn't look the right approach to modify the very base initial config. We've got already a section to specify agent stuff conditionally, so eventually we should insert the changes there rather than duplicating. Wondering also why we need so many resources for the conformance tests, but we can discuss it separately
The CI conformance tests need more resources than currently configured for installations. Allow these settings to be overridden. This requires a corresponding change in openshift/release to use higher limits.
bfournie
force-pushed
the
allow-agent-overides
branch
from
567b3d4 to
6d54ebc
Compare
|
/test e2e-agent-sno-ipv6 |
|
@bfournie: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
2 participants
The CI conformance tests need more resources than currently configured for installations. Allow these settings to be overridden. This requires a corresponding change in openshift/release to use higher limits.